-
Notifications
You must be signed in to change notification settings - Fork 149
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feature: Adding CollectionGroup queries #578
Conversation
4d4ff12
to
9fae4c7
Compare
9fae4c7
to
b2d9fb2
Compare
b2d9fb2
to
9498764
Compare
Codecov Report
@@ Coverage Diff @@
## master #578 +/- ##
==========================================
- Coverage 95.73% 95.66% -0.07%
==========================================
Files 24 24
Lines 1969 1984 +15
Branches 167 167
==========================================
+ Hits 1885 1898 +13
- Misses 63 64 +1
- Partials 21 22 +1
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #578 +/- ##
======================================
Coverage 55.1% 55.1%
======================================
Files 5 5
Lines 274 274
======================================
Hits 151 151
Misses 123 123 Continue to review full report at Codecov.
|
readonly parentPath: ResourcePath, readonly collectionId: string, | ||
readonly allDescendants: boolean, readonly fieldFilters: FieldFilter[], | ||
readonly fieldOrders: FieldOrder[], readonly startAt?: QueryCursor, | ||
readonly endAt?: QueryCursor, readonly limit?: number, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like startAfter
and endBefore
were omitted, what is the reason? Looks like they were unused previously?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is actually meant to mimic the proto, where startAfter
and endBefore
are just different settings in the QueryCurso (startAt
sets before to true
, whereas startAfter
sets it to false
).
I'm really sorry, I somehow managed to miss this PR entirely. I usually respond no later than the next day on code reviews; feel free to ping me if I don't. |
CollectionGroup queries add support for queries to Firestore that query documents by collection ID rather than by collection path. With this feature, users can now query for documents in collections that share a name, no matter where these collection are located.
This the Node port of googleapis/google-cloud-java#4652. The integration tests are ported from firebase/firebase-js-sdk#1440
Note: This PR is blocked on a backend release.